Skip to content

Create enable-in-memory-oltp.sql - #23

Closed
Jos de Bruijn (jodebrui) wants to merge 1 commit into
masterfrom
jodebrui-patch-3
Closed

Jos de Bruijn (jodebrui) wants to merge 1 commit into
masterfrom
jodebrui-patch-3

Conversation

@jodebrui

Copy link
Copy Markdown
Contributor

Sample script that enables the current database for In-Memory OLTP, including best practices for database options

Sample script that enables the current database for In-Memory OLTP, including best practices for database options
Umachandar Jayachandran (uc-msft) pushed a commit that referenced this pull request Aug 31, 2026
The integration runtime filter selected any runtime with Type 'Managed' whose
LicenseType differed from the target. The default AutoResolveIntegrationRuntime
is also Type 'Managed' (it is the data-flow runtime, not an SSIS-IR) and has a
null LicenseType, so 'null -ne LicenseIncluded' selected it. The subsequent
Set-AzDataFactoryV2IntegrationRuntime round-tripped the full payload and the
service rejected it with:

  Conflict / DataFactoryPropertyUpdateNotSupported
  Updating property managedVirtualNetwork is not supported

The filter now also requires a non-empty LicenseType, which only SSIS
integration runtimes carry.

The failure was additionally reported as a success: the cmdlet had no
-ErrorAction Stop, so the Conflict was non-terminating, the enclosing catch
never fired, and the script printed '-- DataFactory ... updated to license type
LicenseIncluded' directly after two error blocks. Each runtime is now updated
inside its own try/catch with -ErrorAction Stop.

Applies the same result-checking to the SQL VM path, which piped 'az sql vm
update' straight into ConvertFrom-Json without checking LASTEXITCODE and
appended its CSV row before the attempt.

Rows are now projected onto an explicit column set at export time, because
Export-Csv derives its header from the first object only and would otherwise
drop UpdateResult/UpdateError depending on which section emitted first.

Adds TESTPLAN cases #23-#26 and re-syncs the embedded copy in the
self-contained orchestrator.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant